-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: 개발 서버를 추가하고, 워크플로우를 개선한다 #126
Conversation
- 사용 가능한 포트가 없는 경우 대응 - 배포 실패를 확인할 수 없는 오류 수정
- 스크립트에는 환경 변수 이름 변경을 반영한다
@@ -0,0 +1,126 @@ | |||
name: 변경사항을 운영 서버에 배포한다 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 스크립트는 dev-deploy.yml
과 발동 조건 빼고 동일합니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그 발동 조건이 branches: main
이고 job
runs-on 부분만 다른거 맞죠? 확인차 여쭙니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그 발동 조건이
branches: main
이고job
runs-on 부분만 다른거 맞죠? 확인차 여쭙니다.
맞습니다!
@@ -0,0 +1,49 @@ | |||
name: 승인된 변경사항을 운영 브랜치에 반영한다 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이건 아직 정상 동작하는지 모릅니다.. 실제 돌려봐야 알 수 있겠네요 ㅋㅋ
문서 일일이 찾아가며 고생하셨습니다! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!
PORT_A_PID=$(lsof -ti:${{ vars.APPLICATION_PORT_A }}) | ||
PORT_B_PID=$(lsof -ti:${{ vars.APPLICATION_PORT_B }}) | ||
if [ -n $PORT_A_PID -a -n $PORT_B_PID ]; then | ||
echo "::error title=배포 실패::$PORT_A_PID, $PORT_B_PID 두 포트가 모두 사용중입니다"; | ||
exit 1; | ||
fi | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
포트를 둘다 쓰고 있을 경우가 있나요??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오류가 난 적 있었어요. (짧은 시간에 여러번 배포시도한 경우)
스크립트가 서버 잘 실행된줄 알고 계속 접속 시도하더라구요.
.github/workflows/dev-deploy.yml
Outdated
-DPRODUCTION_DB_URL=${{ secrets.DB_URL }} \ | ||
-DPRODUCTION_DB_USERNAME=${{ secrets.DB_USERNAME }} \ | ||
-DPRODUCTION_DB_PASSWORD=${{ secrets.DB_PASSWORD }} \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DPRODUCTION_
이라는 이름이 '운영 서버'의 의미로 썼던건가요 아님 다른 의미가 있나요??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
운영의 의미로 사용했습니다.
이제 환경에 따라 프로파일 및 시크릿이 분리되었기 때문에 PRODUCTION_
접두사를 유지할 필요가 없어서 제거했습니다.
@@ -0,0 +1,126 @@ | |||
name: 변경사항을 운영 서버에 배포한다 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그 발동 조건이 branches: main
이고 job
runs-on 부분만 다른거 맞죠? 확인차 여쭙니다.
고민한 사항 공유
노션 링크입니다.
곧 스낵게임 문서로 복사해 넣겠습니다.
변경 사항
AS-IS
TO-BE
(개발 서버 및 DB 세팅 완료)